fix: allow to use this theme an old php version #737
Conversation
inc/actions.php
Outdated
| function dsi_schede_progetti_filters( WP_Query $query ) { | ||
|
|
||
| if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || (get_queried_object()?->taxonomy ?? null) == "tipologia-progetto") ) { | ||
| if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || (get_queried_object() && isset(get_queried_object()->taxonomy) && get_queried_object()->taxonomy == "tipologia-progetto")) ) { |
There was a problem hiding this comment.
Segnalo che WP ha anche una funzione is_tax(), che è più compatta e "idiomatica" e svolge già tutti i controlli su get_queried_object().
| if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || (get_queried_object() && isset(get_queried_object()->taxonomy) && get_queried_object()->taxonomy == "tipologia-progetto")) ) { | |
| if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || is_tax("tipologia-progetto")) ) { |
[v. #691]
There was a problem hiding this comment.
qui si è modificata la linea precedente, in quanto la versione presente andava in conflitto con versioni di PHP precedenti all'8.0
There was a problem hiding this comment.
Sì, infatti: è l'operatore nullsafe ?->, che non è supportato.
Ma di tutta la sequenza delle verifiche su get_queried_object->taxonomy credo che se ne potrebbe fare a meno, perché quello è proprio il mestiere di is_tax().
|
|
||
| // If 'with_front' is true, dd $wp_rewrite->front to the trail. | ||
| if ( $taxonomy->rewrite['with_front'] && $wp_rewrite->front ) | ||
| if ( array_key_exists('with_front', $taxonomy->rewrite) && $taxonomy->rewrite['with_front'] && $wp_rewrite->front ) |
There was a problem hiding this comment.
Se deve essere controllata l'esistenza, perché non semplicemente ! empty()?
There was a problem hiding this comment.
ci potrebbero essere dei casi in cui with_font non era valorizzata, ecco perche questo check in più
There was a problem hiding this comment.
Sì; intendevo che ! empty($taxonomy->rewrite['with_front']) ha lo stesso effetto ed è più leggibile e conciso.
Co-authored-by: enrimk <[email protected]>
zetareticoli
left a comment
zetareticoli
left a comment
There was a problem hiding this comment.
Cosa intendi con versioni antecedenti alla 8, PHP?
|
7.3 e 7.4 |
| foreach ( $post_types as $type ) { | ||
|
|
||
| if ( $slug === $type->has_archive || ( true === $type->has_archive && $slug === $type->rewrite['slug'] ) ) | ||
| if ( $slug === $type->has_archive || ( true === $type->has_archive && is_array($type->rewrite) && isset($type->rewrite['slug']) && $slug === $type->rewrite['slug'])) |
There was a problem hiding this comment.
Non è un'eventualità piuttosto remota?
Cioè, mi pare che rewrite['slug'] qui possa risultare non definito solo nel caso di un post_type con has_archive === true e rewrite === false contemporaneamente (con archivio, ma senza permalink). È così? È un caso che si verifica nell'app?
Co-authored-by: enrimk <[email protected]>
4 participants
Descrizione
le modifiche fatte consentono di far funzionare il tema anche su versioni antecedenti alla 8.
Modificate alcune funzioni
Aggiunta una funziona prima che sia deprecata l'originale